Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ignores_find_in_page to inert global attribute #25984

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Feb 21, 2025

Summary

Adds subfeature ignores_find_in_page to html.global_attributes.inert, duplicated from api.HTMLElement.inert.

Test results and supporting details

Related issues

Fixes #25983.

Duplicated from `api.HTMLElement.inert.ignores_find_in_page`.
@caugner caugner requested a review from Elchi3 February 21, 2025 08:14
@github-actions github-actions bot added data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML size:m [PR only] 25-100 LoC changed labels Feb 21, 2025
"ignores_find_in_page": {
"__compat": {
"description": "Element is ignored for the purposes of find-in-page.",
"mdn_url": "https://developer.mozilla.org/docs/Web/API/HTMLElement/inert",
Copy link
Member

@Elchi3 Elchi3 Feb 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see any useful docs on find-in-page the HTMLElement page.

Suggested change
"mdn_url": "https://developer.mozilla.org/docs/Web/API/HTMLElement/inert",
"mdn_url": "https://developer.mozilla.org/docs/Web/HTML/Global_attributes/inert",

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At second glance, shouldn't we remove the mdn_url here and from api.HTMLElement.inert.ignores_find_in_page, given that it refers to the parent feature, which has this mdn_url? Consumers can duplicate the mdn_url down if they want to.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removal also works for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML size:m [PR only] 25-100 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTML inert attribute doc should mention the ignores_find_in_page Safari issue
2 participants